Upgrade TailBlazor progress bar for .NET 9 auto render#1
Upgrade TailBlazor progress bar for .NET 9 auto render#1TaylorWatson wants to merge 1 commit intomainfrom
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting
| <ItemGroup> | ||
| <SupportedPlatform Include="browser" /> | ||
| <FrameworkReference Include="Microsoft.AspNetCore.App" /> | ||
| </ItemGroup> |
There was a problem hiding this comment.
Switching the project to net9.0 replaced the Microsoft.AspNetCore.Components.Web package with a FrameworkReference to Microsoft.AspNetCore.App. Framework references are only valid for server-hosted components and aren’t supported when the library is referenced from a net9.0-browser or Blazor WebAssembly project. Because Auto render mode needs to load the same component assembly in both server and browser contexts, this change prevents the package from being consumed by the client half of an Auto app and breaks the commit’s stated goal. Consider keeping the package reference or multi-targeting so the browser build doesn’t transitively depend on Microsoft.AspNetCore.App.
Useful? React with 👍 / 👎.
1 participant
Summary
Testing
https://chatgpt.com/codex/tasks/task_e_68e72898bd18833288e0080996cf759b